Skip to content

Conversation

ShourieG
Copy link
Contributor

@ShourieG ShourieG commented Aug 19, 2025

Type of change

  • Bug

Proposed commit message

Fixed system test failures from streaming input based integrations 
and re-enabled system tests for proofpoint_on_demand.

Note

Since this is only related to system tests, changelog and version increments have not been done.
A more detailed explanation of the issue can be found here.

Checklist

  • I have reviewed tips for building integrations and this pull request is aligned with them.
  • I have verified that all data streams collect metrics or logs.
  • I have added an entry to my package's changelog.yml file.
  • I have verified that Kibana version constraints are current according to guidelines.
  • I have verified that any added dashboard complies with Kibana's Dashboard good practices

Author's Checklist

  • [ ]

How to test this PR locally

Related issues

Screenshots

@ShourieG ShourieG self-assigned this Aug 19, 2025
@ShourieG ShourieG requested a review from a team as a code owner August 19, 2025 12:32
@ShourieG ShourieG added bugfix Pull request that fixes a bug issue Team:Security-Service Integrations Security Service Integrations team [elastic/security-service-integrations] Integration:websocket Custom Websocket logs Integration:proofpoint_on_demand Proofpoint On Demand labels Aug 19, 2025
@elasticmachine
Copy link

Pinging @elastic/security-service-integrations (Team:Security-Service Integrations)

@elasticmachine
Copy link

💚 Build Succeeded

cc @ShourieG

Copy link

Quality Gate passed Quality Gate passed

Issues
0 New issues
0 Fixed issues
0 Accepted issues

Measures
0 Security Hotspots
No data about Coverage
No data about Duplication

See analysis details on SonarQube

log.Println(err)
return
}
defer conn.Close()
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Yep. That's your problem right there.

@ShourieG ShourieG merged commit 84fb739 into elastic:main Aug 20, 2025
9 checks passed
@ShourieG ShourieG deleted the bugfix/#14854 branch August 20, 2025 04:48
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bugfix Pull request that fixes a bug issue Integration:proofpoint_on_demand Proofpoint On Demand Integration:websocket Custom Websocket logs Team:Security-Service Integrations Security Service Integrations team [elastic/security-service-integrations]
Projects
None yet
Development

Successfully merging this pull request may close these issues.

[websocket][proofpoint_on_demand] - System tests failure from 8.19 and above
3 participants